Skip to content

Fix Grid cache invalidation for multi-run task operations#55504

Merged
eladkal merged 4 commits into
apache:mainfrom
bggwak:55267
Sep 12, 2025
Merged

Fix Grid cache invalidation for multi-run task operations#55504
eladkal merged 4 commits into
apache:mainfrom
bggwak:55267

Conversation

@bggwak

@bggwak bggwak commented Sep 11, 2025

Copy link
Copy Markdown
Contributor

Problem

Grid UI was not updating after clearing or patching task instances when
using the include_future or include_past options.(#55267 )

Solution

  • Fixed cache invalidation logic to properly refresh all affected DAG runs,
    not just the current run.
  • Considered precise invalidation based on past/future options, but this
    required overly complex predicate logic.
  • Opted for a simpler approach: invalidate all DAG Grid data whenever
    multiple runs are affected.

Related Issues

Fixes #55267

@boring-cyborg boring-cyborg Bot added the area:UI Related to UI/UX. For Frontend Developers. label Sep 11, 2025
Comment thread airflow-core/src/airflow/ui/src/queries/useClearTaskInstances.ts
Comment thread airflow-core/src/airflow/ui/src/queries/useClearTaskInstances.ts Outdated

@pierrejeambrun pierrejeambrun left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code looks good, CI is not happy, you might need use the spread operator instead to set or unset the runid variable.

@eladkal eladkal added this to the Airflow 3.1.0 milestone Sep 12, 2025
@eladkal eladkal added the type:bug-fix Changelog: Bug Fixes label Sep 12, 2025
@eladkal eladkal merged commit 07dd277 into apache:main Sep 12, 2025
57 checks passed
kaxil pushed a commit that referenced this pull request Sep 15, 2025
* Fix Grid cache invalidation for multi-run task operations

* made the code simpler

Co-authored-by: Brent Bovenzi <brent.bovenzi@gmail.com>

* Restore variable name

* fix typescript error

---------

Co-authored-by: Brent Bovenzi <brent.bovenzi@gmail.com>
(cherry picked from commit 07dd277)
suman-himanshu pushed a commit to suman-himanshu/airflow that referenced this pull request Sep 17, 2025
* Fix Grid cache invalidation for multi-run task operations

* made the code simpler

Co-authored-by: Brent Bovenzi <brent.bovenzi@gmail.com>

* Restore variable name

* fix typescript error

---------

Co-authored-by: Brent Bovenzi <brent.bovenzi@gmail.com>
Brunda10 pushed a commit to Brunda10/airflow that referenced this pull request Sep 17, 2025
* Fix Grid cache invalidation for multi-run task operations

* made the code simpler

Co-authored-by: Brent Bovenzi <brent.bovenzi@gmail.com>

* Restore variable name

* fix typescript error

---------

Co-authored-by: Brent Bovenzi <brent.bovenzi@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:UI Related to UI/UX. For Frontend Developers. type:bug-fix Changelog: Bug Fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

UI - Grid invalidation on TI Action

4 participants